Skip to content

(#12357) Fix broken compatibility with Puppet 2.6#49

Merged
kbarber merged 1 commit intopuppetlabs:2.3.xfrom
jeffmccune:bug/2.3.x/fix_absolute_path_error_with_puppet26
Mar 12, 2012
Merged

(#12357) Fix broken compatibility with Puppet 2.6#49
kbarber merged 1 commit intopuppetlabs:2.3.xfrom
jeffmccune:bug/2.3.x/fix_absolute_path_error_with_puppet26

Conversation

@jeffmccune
Copy link
Copy Markdown
Contributor

Without this patch, the previous change set to the
validate_absolute_path() parser function contains Puppet 2.6
incompatible changes. stdlib 2.x is compatible with Puppet 2.6. These
changes are a problem because we cannot introduce backwards incompatible
changes in a minor release.

This patch fixes the problem by back porting the implementation of the
Puppet::Util.absolute_path? from 2.7.x to the function block itself.

The function block tests to see if Puppet::Util.absolute_path? will
respond and if not, falls back to the inline back ported implementation.

The spec tests have been updated to simulate the behavior of Puppet 2.6
even when running with Puppet 2.7.

Without this patch, the previous change set to the
validate_absolute_path() parser function contains Puppet 2.6
incompatible changes.  stdlib 2.x is compatible with Puppet 2.6.  These
changes are a problem because we cannot introduce backwards incompatible
changes in a minor release.

This patch fixes the problem by back porting the implementation of the
`Puppet::Util.absolute_path?` from 2.7.x to the function block itself.

The function block tests to see if `Puppet::Util.absolute_path?` will
respond and if not, falls back to the inline back ported implementation.

The spec tests have been updated to simulate the behavior of Puppet 2.6
even when running with Puppet 2.7.
kbarber added a commit that referenced this pull request Mar 12, 2012
…ror_with_puppet26

(#12357) Fix broken compatibility with Puppet 2.6
@kbarber kbarber merged commit 5d1cec8 into puppetlabs:2.3.x Mar 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants